@xuzhg,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

I'm good with the structure changes and the Microsoft prefix. I'm not a huge fan of the .core suffix. First it creates confusion with .net core and secondly it is sort of one of those generic "manager" terms that doesn't mean a whole lot. I don't feel strongly about it though. Would using Microsoft.OpenApi.Model be better? Is there a specific advantage to having the suffix?

What do you think @PerthCharern ?

Agree with @darrelmiller on the .Core naming. I think .Models (plural?) definitely sounds better.

On that front, does it make sense to move the Writers out to its own folder (instead of as a subfolder of the Models)? It seems more visible that way.

@darrelmiller @PerthCharern I agree that. I will change it.

@PerthCharern That's the plan to move the writer out the "model" after the structure changes.

Sure. I'd prefer to just use Microsoft.OpenApi. I'm ok with the Writers being moved to a new folder. But the plan is it to keep it in the same project/nuget.

@darrelmiller @PerthCharern

Ok. Below is the final structure.

@darrelmiller @xuzhg Actually, ignore my comments about the writers. Not sure what I was thinking.

Agree with @darrelmiller that the Writers should be in the same project/nuget as model, so it makes sense that they are where they are right now (already in a separate folder).

The overall structure looks good to me.

Merged at: 81e3b9b